Skip to content

Fix Supabase Auth#81

Merged
XploY04 merged 4 commits into
pointblank-club:stagingfrom
jayantkageriofficial:staging
Jun 22, 2026
Merged

Fix Supabase Auth#81
XploY04 merged 4 commits into
pointblank-club:stagingfrom
jayantkageriofficial:staging

Conversation

@jayantkageri

Copy link
Copy Markdown
Contributor

Summary: What does this PR do?

Fix the Supabase Authentication Issues due to migrating to SSR
The code_verifier was not being stored on the server side, hence exchangeCodeForSession was returning false
The Session Token Storage is now unified in Cookies, unless previously in localStorage, sessionStorage and also cookies

Which issue(s) this PR fixes:

Changes Made

Describe the changes you've made in this PR:

  • Feature implementation
  • Bug fix
  • Documentation update
  • Code refactoring
  • Configuration changes
  • Other (please specify)

Type of Change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Documentation update
  • Performance improvement
  • Code cleanup or refactor

How Has This Been Tested?

Describe the tests you ran:

  • Unit tests
  • Integration tests
  • Manual tests
  • Other (please specify)

Please describe the test cases and expected behaviour:

  1. Auth System Working Correctly

Screenshots (if applicable)

Dependencies

Documentation

  • I have updated the documentation accordingly
  • Documentation update is not required

Comments:

Reviewer Notes

@netlify

netlify Bot commented Jun 22, 2026

Copy link
Copy Markdown

👷 Deploy request for pb-placement pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 9798797

@github-actions github-actions Bot added Dependencies Auto-generated label for Dependencies changes Frontend Auto-generated label for Frontend changes size-L Auto-generated label for size-L changes Utils Auto-generated label for Utils changes labels Jun 22, 2026
@github-actions github-actions Bot added size-L Auto-generated label for size-L changes and removed size-L Auto-generated label for size-L changes labels Jun 22, 2026
Comment thread app/api/email-sign-in/route.ts
@unichronic

Copy link
Copy Markdown
Contributor

LGTM

@XploY04 XploY04 merged commit a761132 into pointblank-club:staging Jun 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Dependencies Auto-generated label for Dependencies changes Frontend Auto-generated label for Frontend changes size-L Auto-generated label for size-L changes Utils Auto-generated label for Utils changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants